-
Notifications
You must be signed in to change notification settings - Fork 1.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Write log entry when Azure DevOps API Rate Limit is (nearly) reached #6285
base: main
Are you sure you want to change the base?
Write log entry when Azure DevOps API Rate Limit is (nearly) reached #6285
Conversation
Semgrep found 1 Consider to use well-defined context Ignore this finding from context-todo.Semgrep found 1 Detected directly writing or similar in 'http.ResponseWriter.write()'. This bypasses HTML escaping that prevents cross-site scripting vulnerabilities. Instead, use the 'html/template' package and render data using 'template.Execute()'. Ignore this finding from no-direct-write-to-responsewriter. |
Signed-off-by: Ruben Herrebrugh <[email protected]>
95f03d4
to
37630d3
Compare
I haven't changed this code, how should I go about resolving these semgrep issues? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice improvement! Thanks 🙇
Don't worry, I've ignored those findings for the PR and the check should pass now |
/run-e2e pipeline |
Azure Pipelines Scalar
Checklist
Fixes #6284
Not related to other PRs